chore: remove duplicate files, dead hooks, and redundant rules#10
Closed
saif-at-scalekit wants to merge 1 commit into
Closed
chore: remove duplicate files, dead hooks, and redundant rules#10saif-at-scalekit wants to merge 1 commit into
saif-at-scalekit wants to merge 1 commit into
Conversation
- Delete duplicate agent-connectors/ directory (44 files identical to
references/agent-connectors/)
- Merge 4 missing connector entries (Attention, Chorus, Clari Copilot,
Google Slides) into references/agent-connectors/README.md
- Delete 6 duplicate top-level doc files (byoc.md, connected-accounts.md,
connections.md, code-samples.md, providers.md, redirects.md) that were
identical to copies in references/
- Delete hooks/ directories from all 5 plugins (referenced
${CLAUDE_PLUGIN_ROOT}/hooks/beacon.sh which is a Claude Code variable,
not Cursor, and no beacon.sh existed in this repo)
- Delete mcp-auth/rules/no-secrets.mdc (fully redundant with the more
specific mcp-secrets-hygiene.mdc in the same plugin)
- Update AGENTS.md to remove no-secrets.mdc from mcp-auth rules list
This was referenced May 12, 2026
Collaborator
Author
|
Superseded by #12 which includes this cleanup plus the 5→2 AgentKit/SaaSKit consolidation. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Structural cleanup that removes dead weight from the Cursor authstack — duplicate files, non-functional hooks, and a redundant rule.
Changes
Duplicate content removed
agent-connectors/directory (44 files) — identical copies of files already inreferences/agent-connectors/. Merged 4 missing connector entries (Attention, Chorus, Clari Copilot, Google Slides) into the canonicalreferences/copy before deletion.byoc.md,connected-accounts.md,connections.md,code-samples.md,providers.md,redirects.md) — byte-for-byte identical to copies inreferences/.Dead hooks removed
hooks/directories from all 5 plugins — allhooks.jsonfiles referenced${CLAUDE_PLUGIN_ROOT}/hooks/beacon.sh, which is a Claude Code environment variable (not Cursor). Nobeacon.shscript existed in this repo. These were cargo-culted from the Claude Code authstack and had no runtime effect in Cursor.Redundant rule removed
mcp-auth/rules/no-secrets.mdc— generic "no hardcoded secrets" rule fully covered by the more specificmcp-secrets-hygiene.mdcin the same plugin (which lists actual Scalekit env var names).AGENTS.mdto removeno-secrets.mdcfrom the mcp-auth rules list.Validation
node scripts/validate-template.mjspasses. Only informational warnings about missing hooks (expected) and MCP configs (unchanged).Impact
references/locations